box: Move vfunc implementations before class_init

Just like we do everywhere else. This way we can also get rid of all the
prototypes.

calendar: Remove day-selected-double-click signal

It doesn't really make sense to treat double clicks here different than
single clicks (and is bad UX), and it also breaks switching months by
quickly trying to single-click the last/first days in the calendar.

calendar: Remove an unused enum

calendar: Remove month-changed signal

Listen to notify::month instead.

calendar: Keep a GDateTime around

Instead of a year/month/day triple.

calendar: Return a GDateTime from get_date()

calendar: Use GDateTime to select days

calendar: Use widgets for everything

calendar: Try to avoid changing size when changing the year

calendar: Remove spinning code

This is dead code

calendar: Add accessors for display properties

Instead of having a display flags enum and bitfield, just add proper
accessors for the properties, which is what we do everywhere else.

testcalendar: Remove unused function

calendar: Remove details

This is an unused feature that's way too complicated for a default
calendar widget and complicates the implementation a lot. Since we want
to eventually replace this with actual widgets, remove the details
support now.

calendar: Fix an uninitialized value

Values pased to gtk_widget_measure must be initialized to 0. This failed
if the header widget of the calendar was invisible.

Adwaita: Fix levelbar styling

The block nodes are inside a trough node.

{box,grid}layout: Access border spacing directly

Instead of going through the style context

GskTransform: Remove useless fmod check

All angles are in the [0; 360[ range

GskTransform: add assertions to make normalize_angle() intentions  clear

360deg should be expressed as 0deg and no angle should ever be
negative.

snapshot: Remove new_with_parent

Unneeded now that we have push_collect()/pop_collect()

textlayout: Use gtk_snapshot_{push,pop}_collect()

GskRoundedRect: Inline graphene_rect_contains_rect

Brings gsk_rounded_rect_contains_rect down from 0.54% to 0.14% when
rendering rounded backgrounds.

widget: Use gtk_snapshot_{push,pop})_collect

So we avoid allocating a new snapshot just for that.

snapshot: Add _push_collect() and _pop_collect()

These will replace the previous gtk_snapshot_new_with_parent(), which
allocated an entirely new GObject just to push()/pop() some state. This
is already a problem but will be more important in the future as we
start using this more.

Start using GtkWidget's new style class API

renderborder: Make simple border case more obvious

When reaching this point, it is impossible that all border styles are
HIDDEN or NONE, but up to 3 of them can still be that style. In any
case, the "none or solid" border style is the most common on there is,
so try to make this simpler here by just appending a border node
directly instead of going through the snapshot_border path.

Move GTK_BORDER_STYLE_HIDDEN before GTK_BORDER_STYLE_SOLID

The simple case when drawing border is "solid or nothing", which we can
now check for as simple as "style <= GTK_BORDER_STYE_SOLID".

testcalendar: Close window properly

searchbar: Use proper getter in _get_property()

Instead of retuning a seemingly random boolean, just use
gtk_search_bar_get_search_mode().

searchbar: Use g_object_notify_by_pspec

We already keep the pspecs around, so use them.

progressbar: Remove a few unneeded queue_resize() calls

This will happen automatically when adding style classes, setting the
label text, etc.

Window: inline function into only caller

widget: Fix a from_string/try_string mixup

widget: Speed up verify_invariants()

Hide the expensive GTK_IS_ROOT() checks behind the cheaper realized,
mapped, etc. checks. This way we only check for the widget being a root
if the invariant does not hold.

This gets pop_verify_invariants() from 16% to 6% when running the
scrolling benchmark in the widget-factory.

widget: Remove verify_child_invariants

Just calls verify_invariants() anyway.

Update POTFILES.in

NEWS: Small additions

flatpak: Build against the master runtime

This is what the ci is set up to use, and building
against 3.34 fails there.

Merge branch 'wip/otte/toolbar' into 'master'

remove GtkToolbar

See merge request GNOME/gtk!1383

GtkToolbar: remove

testsuite: Get rid of GtkToolbar

Just use GtkBox instead

css: Remove .primary-toolbar

It's not used anywhere anymore.

css: Remove .inline-toolbar

It's not used anywhere anymore.

widget-factory: Get rid of toolbars

gtk-demo: Get rid of toolbars

bloatpad: Use a box, not a toolbar

print: Don't use a toolbar in the custom paper dialog

Use a regular linked box instead.

cssnode: Remove leftover vfunc

Remove accidentally added files

Add a forgotten word

Fix syntax

Updates for 3.98

Merge branch 'wip/carlosg/implicit-grabs-on-leave' into 'master'

Handle implicit grabs on leave events

Closes #13

See merge request GNOME/gtk!1397

gtkmain: Wait for GDK_CROSSING_UNGRAB leave event with implicit grabs

If there is a passive grab and the pointer leaves the window we would
receive a GDK_CROSSING_NORMAL event when the pointer moves outside
the window, and a GDK_CROSSING_UNGRAB event when we do release the
button and the implicit grab.

We currently would react to the first, but want to react to the
second. In the time between both events, the client would still receive
pointer motion that will reach the implicitly grabbed widget.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/13

gtkroot: Make enum a typedef

Seems a typo, and prevents building as GtkRootProperties is deemed
to be defined multiple times.

Merge branch 'a11y-fixes' into 'master'

A11y fixes

See merge request GNOME/gtk!1396

testsuite: Misc. updates of expected output

The a11y dumps for action and header bars
have changed in harmless ways.

infobar: Implement forall

As a container, that is what you have to do.

testsuite: Clean up a11y menu tests

We don't need some of the tests anymore, since
we just have popovers now.

testsuite: Improve --generate support

When using the test binary to generate expected
output, don't initialize the test machinery, since
that pollutes the output with noise.

tests: Fix infobars in ui files

GtkInfoBar no longer exposes the content_area
as internal child.

testsuite: Update expected output

colorchooser: Use a composite accessible

This makes the content show up in the a11y tree again.

tests: Update expected output

The stackswitcher is no longer a box, so it doesn't
have the horizontal state anymore.

testsuite: Update some a11y tests

GtkMenuButton::use-popover no longer exists, remove it
from test files.

stackswitcher: Use a composite accessible

This makes the buttons show up in the a11y tree again.

a11y: Add an accessible for widgets with children

We've started to turns containers into widgets which
just happen to have children, and some of these need
to be exposed to the a11y stack.

This adds a very minimal implementation, it does not
currently emit change notification when children are
added or removed.

a11y: Report children of widgets

We want children of composite accessibles to be
reported.

Merge branch 'flatpak-build-sassc' into 'master'

Flatpak build sassc

See merge request GNOME/gtk!1392

Build sassc as a dependency for flatpaks

Downloading the subproject during the GTK build fails
in a flatpak build due to lack of network access.
flatpak-builder insists on having these things explicitly
spelled out as dependencies.

Merge branch 'WindowsPrintDriverPreferencesMaster' into 'master'

Fixed bug that prevented GTK from storing printer driver preferences in the…

See merge request GNOME/gtk!397

notebook: Update arrows when text direction changes

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/103

Merge branch 'kill-simple-search' into 'master'

Remove simple search engine

See merge request GNOME/gtk!1393

Merge branch 'better-test-isolation-2' into 'master'

Better test isolation

See merge request GNOME/gtk!1391

css tests: Ignore ".csd" vs ".solid-csd"

It it hard to control which of the csd style classes we get,
since it depends on details of the X server or compositor.

Explicitly ignore this difference by replacing .solid-csd
with .csd in the output.

testsuite: Make tests run without a11y bus

Not having an a11y bus around causes warnings from
at-spi, which make tests fail. Explicitly ignore those
warnings.

testsuite: Fix bloomfilter style tests

The results were expecting solid-csd, which is not what
we usually have.

Merge branch 'master.msvc' into 'master'

Fix building recent GTK master on Visual Studio

See merge request GNOME/gtk!1369

Merge branch 'public-emojichooser' into 'master'

emojichooser: Make public

See merge request GNOME/gtk!1382

Merge branch 'wip/otte/css' into 'master'

Some CSS cleanup

See merge request GNOME/gtk!1389

emojichooser: Add profiler marks for loading

This is a long-running operation, and worth showing
in traces.

icontheme: Fix various memory leaks

Tasks need to be unreffed too.

widget: Fix a memory leak with class actions

This code was doing the wrong thing even if the parent class
doesn't have actions.

Add a lib64 variant of gtk.supp

icontheme: Remove some dead code

frameclock: cosmetics

Redo the profiler marks for the frame clock signals
to look more uniform.

gdk: Silence some compiler warnings

These cause unused variable warnings in release builds.

treestore: Silence compiler warnings

This was causing compiler warnings in release builds.

notebook: Don't trigger criticals in dispose

When we dismantle our children in dispose, we
trigger a11y children-changed signals which end
up calling back into the notebook. Handle this
without critical warnings.

textview: Don't update action state in dispose

Only update the action state when we have a text buffer,
there is no need to do it otherwise, since we are going
to get a buffer before we get shown. This avoids triggering
the action state updates from finalize, which is a bad
time to be recreating the action muxer.

entry: Avoid criticals in dispose

The accessible gets properties of the entry, and
resetting the entry icons triggers accessible change
notification, so do that before we dismantle the entry
too far to respond to a g_object_get () call.

Don't export color picker functions

This is not public API, so it shouldn't be exported.

Read bookmarks async

Do not block the main thread while reading the bookmarks file.
This speeds up initial setup of a GtkPlacesSidebar.

placessidebar: Change some initial property values

Giving these properties the initial value that the file
chooser wants for them avoids some calls to update_places
in the setup code.

testperf: More flexible mark selection

Allow selecting marks not just by their name, but
also by the message detail.

testperf: Allow measuring start time

This can be used e.g. to measure how long it takes us
to get the first frame on screen.

stylecontext: Remove unused typedef

css: Make some local variables const

css: Remove unused function

style context: Remove parent relationship

We don't have foreign drawing anymore, so we don't need this.

stylecontext: Remove gtk_style_context_new()

It's private now, no more foreign drawing of any sort.

To be fair, it was entirely useless without widget paths already.